Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Small errors discovered in English while translating #33

Merged
merged 12 commits into from
Jan 1, 2025
Merged

Small errors discovered in English while translating #33

merged 12 commits into from
Jan 1, 2025

Conversation

ogallagher
Copy link
Contributor

No description provided.

@ogallagher ogallagher mentioned this pull request Sep 24, 2023
Open
1 task
@ogallagher
Copy link
Contributor Author

ogallagher commented Nov 4, 2023 •
edited
Loading

  • latest/ass_tags internal link to #wrapstyle should be #\q
  • latest/ass_tags unescaped backslash at line 57
  • latest/ass_tags internal link to #bordersize should be #\bord
  • latest/ass_tags internal link to #borderwidth should be #\bord
  • latest/ass_tags internal link to #bluredges should be #\be
  • latest/ass_tags other similar internal link corrections
    • #setposition → #\pos
    • #movement → #\move
    • #animatedtransform → #\t
    • #linealignment → #\an
    • #textrotation → #\frx
    • #vectordrawings → #drawing-commands
    • #fontsize → #\fs
    • #fade → #\fad
  • latest/ass_tags other external link corrections
    • styles#thestyleeditor → styles#the-style-editor
    • tutorials#karaoketiming → tutorials#karaoke-timing
  • latest/ass_tags wrap style remove italic escape and add space after end italic

@CoffeeFlux
Copy link
Member

@ogallagher if you want to update this PR, I'm happy to get it merged!

@ogallagher
Copy link
Contributor Author

Alright, will comment if I begin another commit

@ogallagher
Copy link
Contributor Author

Updated @CoffeeFlux

@CoffeeFlux CoffeeFlux merged commit 453feda into Aegisub:master Jan 1, 2025
@ogallagher ogallagher deleted the i18n/en branch January 1, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FichteFoll FichteFoll FichteFoll approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ogallagher @CoffeeFlux @FichteFoll